labofsahil/github-event-dataset-2012 huggingface dataset:
this could be simplifed to
```java
SaslResource.REUSE_SESSION_ATTRIBUTE.parseAndSetParameter(value,saslElement, reader);
```
Why not parameterize these log messages? e.g.
void topologyAttributeDeprecated(String attribute)
Use `if` and spaces.
Yes, we could do some retry and it's no need to notify user.
if (!noteValue) {

}

8)
On 23/06/2012 03:21, Michal Budzynski wrote:
>>     }
>>   
>>     function onMoveEvent(evt) {
>>       evt.stopPropagation();
>> -    var touchPosition = evt.touches ? evt.touches[0].pageX : evt.pageX;
>> -    var difference = initialTouchPosition - touchPosition;
>> -    cardsView.scrollLeft = initialCardViewPosition + difference;
>> +    var touchPosition = {
>> +        x: evt.touches ? evt.touches[0].pageX : evt.pageX,
>> +        y: evt.touches ? evt.touches[0].pageY : evt.pageY
>> +    };
>> +
>> +    if (evt.target.nodeName === 'LI' && MANUAL_CLOSING) {
> className sound way better, I just thought that there is a reason why someone didn't use any before in this part of the project.

Maybe the reason was lazyness ;)
not L22, it's L36
When 'send' is clicked we see in the UI that the input state changes to `disable` (we see all styles of `disabled` there during a small period of time :( ). Would it be possible removing the listener of the button instead?
Obvious, the commentary is not necessary. ;)
No need for the unit when it is 0.
we can make the phone number random *except* for this 2 cases :)
No, I intentionally skip those items that are completely missed, I am waiting on UX feedback this is more my personal preference then anything. If the event is not over yet thought it notifies you.
That's weird. Maybe the platform has changed at some point. Let's land that and try it. @etiennesegonzac You're good to buy some Maté!
International format could also be a 001 949... is that treated as well?
You should add a convenience method that takes in an IReducer and wraps it in ReducedMetric
`get_next_question()` doesn't touch either of these variables. It was moved further down so that when we archive the participant, we archive it before any of the values are reset, otherwise the archived value wouldn't be accurate.
I agree this can be useful since the layout tree and the node tree are not the same.  However, is there a particular reason to make this a FrameworkProperty?  Making it a FrameworkProperty exposes is to the template system such that people could do ugly things like set other content to it, or (even worse) set data bindings to/from it.
Are you referring to the try line? If so yes, I would still recommend it in the event the response fails to be parsed by text/html. And potentially for IE10 future proofing
It looks like there's a lot more state inside the object. Should this use more than just a few parts of it? I can't imagine you would want two relationships between the same models to be considered equivalent if they use different attributes to join on.
function-name prefix
I made these changes. Good catch on pseudo-ring extending an abelian group.

As for the zeroAnnihilates test, I removed it as you suggested, but keep in mind, even if it can be proved from other axioms we are testing, it is still worth testing. This is because we are only doing randomized testing, and the property we are checking may still (with small probability) be false given that the test has passed.  Adding a test for something we can derive from the others could still be a useful thing to decrease the probability we have an error.

That said, given that our tests run with a different seed each time, I think we'd know pretty quickly if we had an error.
can you remove this from the diff, just to keep it a tiny bit cleaner.
Leaving here since this code will be needed for transactions.
You should call this `create_python2_5_start_cmd`, to prepare us for Python 2.7 support.
Can raise `TypeError`.
Is there a better place where to put the function _getUserListLink() because it will be used too by the stats module
Agree and get_*_options have the same issue. will change them into private function. thanks.
Under what conditions might someone want to override this.
The constructor assumes that each codebase in codebases has at least the repository set.. The default value {'':{}} will therefor throw a config error in the constructor OR has to deal with a definition without a repository which is also a special-case.
Indeed, see #464.
I'll admit this has been in the codebase forever and hasn't caused any problems, but I cringe at shelling out to 'tail' instead of implementing this natively.
Oop, that was debugging code for the contrib.namespace lib change.  Removed.
Yeah, seems very reasonable to me too, just ensure that you include it in all our layouts, it's a mess currently :/
What about `2.10.0-RCs`?
Jasmine exports only history for ICQ account
Woah, what the heck have I done there? And yes, good point. Test doesn't really do what it says on the tin.
Can you add a comment on why we're not just falling through (I assume it's because of the AppLayer below).
this should not be part of the final pull request
and these string funcs
ya3
I think this would be better:

```js
if (!isPrimitive(pref.value) || isFloat(pref.value)) { ...
```
Right. That's what I was thinking of. But thinking about again I'm not sure because that would be a big limitation. So just skip this idea please. Someone should simply run 'ant test' first before building and uploading the extension.
The history of `storedTitle`
=============================
Use the force, Luke! ;)

You can follow the blame of `storedTitle` to the inital commit [`c0ed1e99288f`][1] which is equal to [`aedec19990b5`][2] at hg.oxymoronical.com.
You could see, `nightlyApp.storedTitle` was never defined in `suite.js`. But it was in `browser.js`. It was accidentally left out.
After that You would check Philip Chee's [commit][3] (was already linked in the issue #31), specifically the diff of `suite.js`.
You could see, `storedTitle` introduces. :)


Removing the assignment to`defaulttitle`
=========================================
In commit [`3bb023c1c4da`][4] __harthur__ moved away from this initial, `storedTitle` related implementation of "title computing".
After that `nightly.variables.defaulttitle` became a getter. Therefore assigning a value to it is a no-op.
Or just a warning in the Error Console if `javascript.options.strict` is enabled:

	TypeError: setting a property that has only a getter

Examining the commit, You could get the resolution that `nightlyApp.storedTitle` isn't used in the new version.
So it can be removed. (And should be removed in `browser.js`!)

By the way, sorry for the long explanation. :)


[1]: https://github.com/mozilla/nightlytt/blob/c0ed1e99288f/src/chrome/content/suite.js " initial code from http://hg.oxymoronical.com/extensions/NightlyTesterTools/"
[2]: http://hg.oxymoronical.com/extensions/NightlyTesterTools/file/9a147b83afd3/src/chrome/content/suite.js "Added tag release-2.0.2 for changeset aedec19990b5"
[3]: http://hg.oxymoronical.com/extensions/NightlyTesterTools/diff/2c468e058583/src/chrome/content/suite.js "diff of suite.js in changeset 2c468e058583"
[4]: https://github.com/mozilla/nightlytt/commit/3bb023c1c4da#L1L70 "allow default title to occur anywhere in title customization"
the register method is essentially a factory method for drivers.
Btw, I saw many other places where such a comment could have been applied
On Wed, Jul 11, 2012 at 9:35 AM, Andrew Parker
<reply@reply.github.com>
wrote:
>
> If this is safe because it is actually a private method, then can we change the method to be private? Or some other way that makes clear that what you say is the case?

The best I can do is a comment, or using `.send` in all the callers,
since you can't call from an instance to a private class method in
Ruby.

It sounds like you would prefer the duplicate `.intern`, then, so I
will implement that.
Can we include `ScheduleTesting` under the parent describe block to avoid doing it elsewhere?
seems possible that there are a few things worth salvaging in this cron stuff if you didn't already.
Actualy, we do not patch the session object. Just appending event listener https://github.com/riffm/testalchemy/blob/master/testalchemy.py#L77
kill commented code
CS, opening brace is on the wrong line.
This not have sense.
You can add type hinting here `setValidator(ValidatorInterface $validator)`
Doesn't need to be in the standards -- it's simply redundant. :)
I decided to keep the regex for now. It's solid. (BTW, @Maks3w -- yours doesn't take into account the slashes following a scheme, which means you match the server and port as well.)
Leftover pillbox ref here and below
Ah, okay. Then I will delete this.
…and a `res_send(response)` here.?
Do we really want setters?
It's already trimming at new todos, or do you mean something else?
because i replaced filter links with simple text wrapped by <span></span>... 

i had to add them as overrides in app.css because i don't wanted to modify base.css...
whops, pytest/pdbplusplus marker to hide a stack frame, forgot to remove
Align all `=`.
AAAAHHHH!!! I saw an ELSE! Mommy, help me!
Undef warnings.
for the first andalso? I just didn't want to make it too complicated. but if you want, I can add it.
log -> logger (as in the other modules)
and i would use _logger (from vsc.ldap.utils import logger makes no sense imho) (but half of my code probably has the same issue ;)
(and is it used somewhere?)
Please use the header that mentions botth Ghent University and VSC.
You should remove the war plugin configuration, as you *do* have a `web.xml`
I don't think this is true. A redeploy is (should be!) an alias for undeploy and deploy.
http://www.jboss.org/as7

On 1 Apr 2012, at 15:52, sgilda wrote:

>> @@ -170,7 +170,7 @@ The JBoss server can be started a few different ways.
>> The README for each quickstart will specify which configuration is required to run the example.
>> 
>> <a id="startserverweb"></a>
>> -#### Start the JBoss Enterprise Application Platform 6 or JBoss AS 7 Server with the Web Profile
>> +#### Start JBoss Enterprise Application Platform 6 or JBoss Application Server 7 with the Web Profile
> 
> I saw it expanded in the branding image, so I assumed it had to be spelled out.  I've changed everything back to JBoss AS 7..
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jbossas/quickstart/pull/167/files#r630773
Current best way to do `exists` is `var existsSync = fs.existsSync || path.existsSync;`
http://www.python.org/dev/peps/pep-0008/#imports says what @alint says, specifically.
+1 this, good clear naming.
Imports yo,
Changed my mind. this should be:
```python
eq_(u.get_profile().display_name,
    pq(rnv.content)('#profile-info h2').text(),
    'Should be redirected to a user with the right name')
```
I don't know if this is a good fit, a 64bit integer is not representative of a 'byte'.
I haven't had a look yet but feel free to prune code and update the pull request if needed.
It's doable but require more bindings, so it can wait for a bit...
this does not take into account the user allowed hosts, I think you need Host.my_hosts.search ..
lol, I must've been drunk.
Presumably it's possible to jump from 'not live' to 'archived'? If so, deletes to the router and rummager will lead to 404s.

It seems like we should be either changing behaviour to only do those deletions if the item was previously live, or at least testing that those 404s don't lead to exceptions?
I added it to commcare-hq's reqs.  i think for libraries, either listing requirements in the readme or a requrements file is the way to go.
@tchak - and here?
The README is auto-genearted, so this must be removed.
лучше выходить из цикла. тут потенциально может быть ошибка, если в конце что-то дописывать будут.
Shouldn't this be on `Sitemap::Store`? It doesn't seem like it needs to be an instance method on `Middleman::Page`. The less random methods get mixed into that object, the better.
My mistake with regards the the page name, where are we waiting for it to be loaded?
This file can be rebuilt from the asciidoc sources with the Makefile, see if you can't do that, I think there's some date kruft that updates with it.

Thanks for all this!
Does this have identical behavior to directlyOn(shadowedObject)? Kinda confusing...
If we do `getDownloadLink` inside of the `getLatestRevision` call, then it doesn't need to be exposed publicly and will be permission protected.
_getComment should just return a single comment, so doing [0] shouldn't be necessary.
Same comment as above, but in this case ``!isFirst`` should be enough.
there should be a space between the parameters here.
If we let the Repository choose what its criteria is then I guess its the repositories responsiblity to throw an Exception, e.g. `ResourceNotFound(''No routes found for host $host with url $url in locale $foo");`
accidentally committed. should i remove this?
this also doesn't match the assertion you're making below of === false
It's not working if i remove other zend components...
Zend packages does not have their own composer.json file.
Dit kan herschreven als:

```js
if (this.spotted) {
  this.spottedRight = this.pos.x < maincharacter.pos.x;
  this.spottedLeft = this.pos.x > maincharacter.pos.x;
}
Помимо отрицательного числа, вы забываете, что по заданию percent - процентный показатель. Число для замены ещё нужно получить.
ДА)))) Это что то я загнул)))
Во внутренних методах всё же правильнее менять именно значения переменной, чем обращаться к синтезированным сеттерам и геттерам. В последних версиях Xcode synthesize делается автоматически, так что при свойстве с названием dna обратиться к самой переменной можно как к _dna. Ведь свойство у вас может быть readonly, чтобы извне не смогли повлиять на вашу переменную. Как же тогда менять её значение (во внутренних методах, разумеется)
Thanks, I was looking at this yesterday and was thinking of simplifying
in the scope
Yes, looks like breaks are missing.
Can this be moved into ApplicationController or a module? Also it should have a name.
The http:// is not added by "get_abdolute_url()". It is added by the django sitemap module. I will send you a screeshot of the file produced when the default "location" method is used.
Robots will not look for images that are not used/linked anywhere, so excluding them is not necessary.
raw.copy?
I would do:

indices = np.asarray(indices)
That was quite lyrical --- so nitpicking is just what we need before we
release the next mne-python.


On Wed, Dec 12, 2012 at 1:21 AM, Eric89GXL <notifications@github.com> wrote:

> In mne/fiff/raw.py:
>
> > @@ -452,6 +452,7 @@ def apply_hilbert(self, picks, envelope=False, n_jobs=1, verbose=None):
> >      @verbose
> >      def filter(self, l_freq, h_freq, picks=None, filter_length=None,
> >                 l_trans_bandwidth=0.5, h_trans_bandwidth=0.5, n_jobs=1,
> > +               method='fft', iir_params=dict(N=4, ftype='butter'),
>
> I'd say nitpick all the nits there are to pick. We will all have to live
> with accepted changes (not to mention the end-users!), so it's better to
> annoy the fewest people in the long run -- even though it might mean
> annoying the committing coder in the short run :)
> … <#13b8c7d14a79cba6_>
> On Tue, Dec 11, 2012 at 4:15 PM, Martin Luessi <notifications@github.com>wrote:
> In mne/fiff/raw.py: > @@ -452,6 +452,7 @@ def apply_hilbert(self, picks,
> envelope=False, n_jobs=1, verbose=None): > @verbose<https://github.com/verbose>> def filter(self, l_freq, h_freq, picks=None, filter_length=None, >
> l_trans_bandwidth=0.5, h_trans_bandwidth=0.5, n_jobs=1, > + method='fft',
> iir_params=dict(N=4, ftype='butter'), PS: I was thinking about commenting
> on this too.. but then I felt bad about nitpicking too much ;) — Reply to
> this email directly or view it on GitHub<
> https://github.com/mne-tools/mne-python/pull/249/files#r2387100>.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/249/files#r2387154>.
>
>
Lucene provides a few FieldBridges for basic types like Integer, Long, Boolean, etc, but not for Locale.

Lucene does not call toString() on any object by default, which makes sense as toString() is mostly for debug and rarely contains something you'd like to index ;)
Try with:
{code}
mergedData.setPriorDateOfDeathEstimated(preferred.getDeathdateEstimated());
if (preferred.getDeathdateEstimated() == null)
       preferred.setDeathdateEstimated(notPreferred.getDeathdateEstimated());

mergedData.setPriorDateOfDeath(preferred.getDeathDate());
if (preferred.getDeathDate() == null)
       preferred.setDeathDate(notPreferred.getDeathDate());
{code}
github_api gem adds this for us via multi_json gem dependency
there should be a comma after each array item.
yeah its not super clear: https://github.com/thoughtbot/guides/tree/master/style#formatting. Might be worth adding so we have a policy.

Here is loosely what I've been following:
```ruby
  # 1 argument
  say 'hello'

  # 2+ arguments
  say 'hello', 'goodbye'
  
  # nested method calls
  say all_caps('hello')

  # multi line arguments
  say(
    'hello',
    'goodbye',
    etc
  )
```
Is the issue here that you're looking for an HTML fragment and `have_content` only checks against the text of the document?

You can use `page.body.should include(fragment)` for that situation.
Weren't these checks dropped in MW?
> def create pathname, options={}
> make options.merge(:create_if_missing => true, :error_if_exists => true)

It looks nice. I'll change it.
If unstashAll throws an exception, if for instance the mailbox is bounded, then preRestart will throw an exception....
I see. What if you remove the ```layout: "fit"``` default instead of adding ```autoHeight: true```?
I think `itnlimit` is called `maxiter` in the other iterative routines.
Поток же!
I'm just commenting here while I remember. Would't ```self.__class__.__name__``` give you the ```snpSearch``` you're loking for? 

###### Concept
1. We get rid of the subfolders in the template-folder
2. Naming of controllers is as follows: ```controllers/mainController.py``` --> ```controllers/main.py``` etc.
3. Each subclass of ```AppRequestHandler``` ex:  ```snpSearch(AppRequestHandler)``` is unique across the global name-space (as they are now)
4. infer the template-name of the given function as follows

```python
def out(self):
  if self._template == None:
    subclass = self.__class__.__name__
    self._template = subclass + "html"
```
You destroyed the alignment
Maybe a test $iso_code != false ?
Yes it should:

http://code.google.com/p/drmaa-python/source/browse/examples/example5.py
I don't think you need the initialize method here as we're not seeing up any instance variables or default values.  Good thought, though!
please refactor this code above
I locked nanoc version, I forgot I can do that myself :P
yes. I think it was a django change.

Dave
On Sep 6, 2012 12:51 PM, "David Fraser" <notifications@github.com> wrote:

> In sjsoft-j5/settings.py:
>
> > @@ -68,6 +68,7 @@
> >      'django.middleware.common.CommonMiddleware',
> >      'django.contrib.sessions.middleware.SessionMiddleware',
> >      'django.contrib.auth.middleware.AuthenticationMiddleware',
> > +    'django.contrib.messages.middleware.MessageMiddleware',
>
> Is this bit upstream?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/davidfraser/codespeed/pull/1/files#r1543016>.
>
>
more unit tests are needed to prove that the cancelation actually take place,  i.e. interrupt is invoked remotely. You can even create a task that waits forever on an e.g. latch. Then when interrupt is received it stop waiting and increases a static counter.
Maybe remove KNOWN_KEY from the set ?
I don't think this line does as you intended:

```
>>> 'a' in ('hello world')
False
>>> 'hello' in ('hello world')
True
>>> 'w' in ('hello world')
True
```

Adding a trailing comma to turn the thing into a tuple was probably your intention:

```
>>> 'w' in ('hello world', )
False
>>> 'hello world' in ('hello world', )
True
```
docstring needs adding back in.
Do we want to put in a safeguard here from getting stuck in an infinite loop? Is that a likely occurrence?
It's a little bit confusing for me, if I start a new game I would prefer to write the label of the game (ex: Carcassone) instead of the name of the 'partie' (ex: Carcassone, 11/05/12).
It's not about multiple tile managers sharing the same map, but sharing cached tiles. The use case in mind is pre-buffering of time sequences in an off-screen map. Note that only one map can use a cached tile at a time. If another map needs the same tile, a new image element will be created.
Consider inverting this boolean logic so you don't have to wrap your whole function in an if statement.

Like so:
````
if ( ! filter_var($user_id, FILTER_VALIDATE_INT) ) {
  $_SESSION['errors'][] = 'The format of the user ID that was entered is incorrect.';
  return false;
}//end if

// do the rest of your stuff.
````
Might refactor this with some predicate macros so the logic is clearer.
For me CMath::qnorm seems to have a little bug:

in CMath::qsq the summation is done in the following way:
					result+=CMath::pow(x[i], q);

while it should be:
					result+=CMath::pow(fabs (x[i]), q);

aka an absolute value is missing there....
The most common use case is m_q=1. While I would keep q!=1 support, I would just speed up the most common q=1 case with the covertree.
why do you remove progress / gap computation?
If the experiment doesn't have an entry `enabled` in the db, str will be nil, which we rely on to say that experiments start off disabled.
A comment would be nice here.
Agreed, good catch!
well then. i must say. rock on.
Are you sure that the method "plugin()" exists?
Should the type for parameter ``$options`` not be "array|Traversable|null" instead of "array|Traversable"?
Should the type for parameter ``$name`` not be "string|null" instead of "string"?
According Ti.Stream.read - It was the original code from Appcel. so I haven't changed there anything. This is done because buffer size is only 1024 and if message length is more - reading process should be continued.
For the BB this Stream.read is replaced with Stream.pump (which is implemented). And for pump as it seems from doc this is not needed, so in the pumpCallback removed call to the pumpCallback.
Store benötigt noch den Parameter ``Representation rep``...
that wasn't me!! :) but i'll change it
How is

: _1 (true) / 0 (false)_ of whether ...
is it really necessary?
Is this really needed? We don't want eval.
This seems like a pretty specific exception. Should we make our own rather than using a javax one?
Sure, if you prefer.
I changed to focus event but I don't understand what you mean with first comment...
Oh, we had a discussion about this already once. ;-) In English is *same*, which is *isto*. If it would be *equal* above, then we would use *enako*.
Yes, please help me with parsing - example of XML: http://findyourfate.com/rss/dailyhoroscope-feed.asp?sign=Aries
Tnx
You could move date and forecast parsing to two methods. They get XML object (already) parsed and return values.
Please add a integration test for this method.
Good point.  I updated the pull request with this change.
magic numbers!

use enum
Hmmm...should be more specific that they are separate combats. That way it's not great for Phase 1, but might be good for Phase 2. Also picked 3 combats someone gets left out.
please add the <p> tags for proper javadoc formatting
I don't get what's the problem here.
As the plugins.rst is about developer your own plugin, while those are configuration options, I felt it belongs more here than there. Sure we could use our own file about that, but that felt like a structural change in the docs I didn't feel confident to make on my own. But I see your point, especially when there will be more plugins at some point. But for now I feel this is fine enough.
nit: Starting Selenium Grid hub
Also, you now never nullify `pending`. This is a bad practice, memory-wise.
Drop the else and return rc?
It's converting pref value to megabytes
Okay works for me. Have a point in time place to build packages from never
hurts so +1 from me.

Merging.

On Wed, Dec 19, 2012 at 11:04 AM, Greg Sutcliffe
<notifications@github.com>wrote:

> In debian/precise/stable/foreman/build.sh:
>
> > @@ -1,7 +1,7 @@
> >  PACKAGE_NAME='foreman'
> >
> >  REPO='git://github.com/theforeman/foreman.git'
> > -BRANCH='1.0-stable'
> > +BRANCH='89d3f79b13c8a12dac5d1c9b71f6b42176102f7b'
>
> I'm leaning away from using branches for anything other than nightly,
> since a branch's endpoint changes with time - if we were to check this
> specific code out again in a year and try to build the same 1.0.2 package,
> we would get something different (assuming new stuff was commited to the
> branch). Using a tag or sha means the specific version of the package is
> not time-dependant.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/theforeman/foreman-rpms/pull/27/files#r2465112>.
>
>
We are using semantic versioning to a certain extent.  Whenever we release a non-trivial version, I try to increase the "minor" part of the version number.  This PR alone probably warrants a bump to 1.6.0, but with collation it _definitely_ should be increased to 1.6.0.  I'll make the appropriate changes.
no need to put the UUID here as it is retrieved from the database and excluded from the binding anyways
That's a nice idea, I'll fix that here and also in the `OpticalPSF` which takes an interpolant kwarg....
Should we assume users will know what a ring test is?
Same question about x vs. r here as above (and also below on ```__gamma```).  Plus I think the dox for x should say "Radial coordinate x=r/rs" or "Radial coordinate in units of rs" rather than "Radial coordinate in units of r/rs".  Again I can make this change if you agree or don't have some other reason for preferring r here.
Fixed it, sry about that.
I verified on a prod machine that "tempfile" is there
hmmm ... the more we add to this interface the more work it becomes to implement.
also breaks existing classes.  it is a good addition however.

what about standard driver subclasses delegating to stop() (with a warning) ?
Ah, OK. I just cargo-culted it in there.
I think we should add the content-type we want as a predicate for the routes when we add them into pyramid.

As you pointed out, we also need to find a way to capture that the response code isn't valid. We're currently doing so with the special 404 view that we are catching up. I think we can then have a look in the service to see which are the accepted content types and see if this is the problem.

Would that work for you?
OK, it's time to split this file up.
This strikes me as a bit weird---you connect the finished() signal to the listening room's created() signal (implying it is created when the dbcmd is finished) but then you call reportCreated() immediately. If reportCreated is really "add to source" or something, maybe it's better to rename it, otherwise, why would we report it being created before we actually "report it being created" (by emitting the created() signal)
Yea I just copy pasted one of the other error templates and changed it a bit. Sorry :)
Should be a blank line between the last `@param` and `@return` (with the exception of the asterisk).
Please remove unused items from the template before submitting.
unique: true
Ca sert à quoi ?
I's kind of stupid, but I did some system calls just to add some extra spaces at the begining of the lines. To indent a bit and have a _nicer_ output.

For example, I do it `mkdir` with `run_quiet` instead of the `FileUtils` method to avoid non-indented line `mkdir cookbooks`.

I got some different changes like that. I'm not sure if make any sense at all, but I think I'm gonna keep them by now.
how could I not do this after all :smiley:
А дальше эта переменная используется как? calendar.getTime(). Вместо этого можно просто в df.format передавать new Date()
Преобразование к нижнему регистру - достаточно длительная операция. Правильнее использовать String.CASE_INSENSITIVE_ORDER.
Вместо new String(tempExpression) - tempExpression.toString()
Not sure if it could be achieved in a nice way.

I could possibly check for the msFullScreen and oFullScreen flag presence but then it does not guarantee that browser has a functionality to switch video into the full screen mode. In order to check if browser can do it we need to examine video element on the page. Which means most likely in VideoPlayer final init we will check for the existence of RequestFullMode function and then create a feature hasFullScreenMode and merge with fluid.staticEnvironment. All that in finalInit. Is it a decent enough way of doing this ?
A comment for this file would be good.
any reason not to cache the contentDocument as well?
たしかにそっちのほうがよさそうです！
Are we going to put the re-license in for 1.4.1?
you also used single quotation marks [above](#L2R54) - should do the same here if you keep this
exactly! thanks @skade
Please don't had this kind of separators in the code base.
s/predict/transform/
M'kay. `:new_branch => '#{branch}'` should just be `:new_branch => branch`. What you have says that this will *always* use a branch called `#{branch}` (those funny characters included). A single-quoted string is taken literally. A double-quoted string is interpreted. This means that variables inside it will be expanded. That's what those funny characters mean. They mean "replace me with the value of this variable", in this case `branch`. So double quotes would have worked. But since all we need is the value of `branch` we can just use that as a plain 'ol variable.
make currentScreen private
This is redundant code, because `getDsymbol` catches `TOKvar`.
Leftover from android stuff. I'll remove it.
Agreed.  RZCacheSchema is now a base class that RZFileCacheSchema is a subclass of.
RubyMine autoformating sucks: I have ```private``` right under ```class```.

I hate programs, smarter, than me.
no, your patches need backporting, and it's complex as not all these files don't exist in old branch. You should really isolate your changes
implentation > implementation

We usually avoid referring to the impl. code, because it's the "impl" package.. hence quite a black box for users, and because we might move packages around in future. But I have no strong feeling against this, this is actually making it a very interesting read.
of course... it didn't use to recur...
We may 'paging' subfiles (a separate issue) when a 'large' superfile is displayed. From the display, only limited number of subfiles can be selected.
could use else
This is the case where you have an unsigned constant - not a case of it overflowing - so it should be adding a U.  I'm not sure the test

(unsigned __int64)val > INT_MAX)

is correct though.  I think it is avoiding adding the U if the unsigned value is within the signed range, but I suspect it should always be adding a U.  Otherwise the type of an unsigned expression might get changed to signed.  If so both the gcc and vs branches should change.
Oh so do a report_index_base? Yeah I think that makes sense, given how many mini-pages there are turning out to be...
I honestly consider `data` as "local" –it's definitely not remote. It's useful especially in unit test (see all our tests where we pass `data:` instead of real files). If we don't include that scheme, our unit test will be broken.
Plenty of hard-coded column indexes here too - this could break easily.
jshint suggests this? weird :s
Adds a declaration for the unload function to address issue #116.
please remove these merge comments
This is the standard array union operator. I prefer this because it is cleaner. If all keys are strings it works the same.
http://php.net/manual/en/language.operators.array.php
String#ord() isn't going to work in Ruby 1.8, and the variable name of 'i' makes it super confusing anyway. Can I suggest:

````
item.each_char do |c|
   print_status("\t\tValue: #{c}\tCharcode: #{c.unpack("C")}")
````

If you can test this to be sure it works, I think we're good to commit.
elsif res and res.body =~ /names.nsf\?Login/
About the store_loot topic. I think the more correct way to do it is:

1) Save all the configuration files parsed as loots (such as the my.ini file for mysql). In this way people can check where data comes from.
2) Save all the found databases in just one loot file. Isn't very nice to have 5 loot results with just one line.
The phrasing is perfect as is, I just meant "2d" is written "2-D" or "2-dimensional" or "2D".
Remove `scalingFactor`.  Less is more - a lot more.
I'm not entirely clear on what the purpose of the camera's transform is.  Is it defining a local reference frame for the camera?  If so, why does the Columbus View viewExtent need to take it into account but the 3D one does not?  Actually, as I look at this more, it looks like in CV the transform is what makes the horizontal direction the Y coordinate and the vertical direction the Z coordinate.  It makes sense that you need to do that transformation, but if the camera had other transformations appended would you want those too?  I guess you would, but then you would in 3D, too, right?
Good.  Maybe a newline this time?  Only if there is a url, of course.  And maybe more letters shown in the urlizetrunc.  Something like:
```
<dd>
  {{ term.definition|safe }}
  {% if term.url %}
    <br/>
    {{ term.url|urlizetrunc:40 }}
  {% endif %}
</dd>
```
Maybe I'm misunderstanding, but why does this example (which appears to be a CMIS client application that talks to a CMIS web service) need to have dependencies on the server-side libraries? If we add the ModeShape CMIS web service to the AS7 kit, then could this example just deploy the kit (perhaps via the "jboss-as" Maven plugin; see [MODE-1695](https://issues.jboss.org/browse/MODE-1695)) and simply depend on the Chemistry client library (or libraries)?
Android does wrap the views, but as you said, it starts placing them in the center. In addition, it doesn't treat 'top' and 'bottom' the way iOS does--iOS treats 'top' and 'bottom' relative to the *row*, while Android uses absolute 'top' and 'bottom' values. So if you specify top or bottom on Android and the views are wrapped, the 'top' or 'bottom' values move the views out of their rows and you have complete pandemonium.

If Mobile Web is moving to follow iOS, I'll file a bug for Android's behavior, because I don't think it's kind of broken.
Looks like you batch the commands, so it should be fine.  After some discussion, we're going to stick with using delete/insert instead of update.
Debugger statement left in.
Android doesn't have the twofingertap event
Looks like you missed a formating space.
i meant by this as explanation to why there is no auto detection for it. It might be rephrased or removed though to avoid confusion
Like the Ruby compiler, the decision to fork out the HAML stuff was fairly easy.

Firstly, the features required for SASS are very different to those required for HAML, so supporting HAML as well was really not required.
Secondly, it's a case of dogfood - we simply don't use HAML and I don't know anyone in the PHP community who do (Symfony, perhaps?)

With both those reasons considered, it makes much more sense to have a singular focus on great SASS support and let someone else create a great HAML/YAML parser.
Kind of off topic, but are we putting spaces after `foreach` or `for` now? I *wanted* to do it, but haven't seen it done anywhere else before, so didn't want to take that kind of initiative...
nit: i'd put a newline before this
I think it is time to look at BVAL-284 and BVAL-285 in this context again
Well, how would this qs run on openshift?

On 2 Aug 2012, at 23:18, Jason Porter wrote:

>> +            </dependencies>
>> +        </profile>
>> +
>> +        <profile>
>> +            <!-- An optional Arquillian testing profile that executes tests in a remote JBoss AS instance -->
>> +            <!-- Run with: mvn clean test -Parq-jbossas-remote -->
>> +            <id>arq-jbossas-remote</id>
>> +            <dependencies>
>> +                <dependency>
>> +                    <groupId>org.jboss.as</groupId>
>> +                    <artifactId>jboss-as-arquillian-container-remote</artifactId>
>> +                    <scope>test</scope>
>> +                </dependency>
>> +            </dependencies>
>> +        </profile>
>> +        <profile>
> 
> Ah, the openshift one. I initially didn't see the point, but we are trying
> to push openshift, aren't we?
> 
> On Thu, Aug 2, 2012 at 4:15 PM, Pete Muir <
> reply@reply.github.com
>> wrote:
> 
>>> +            </dependencies>
>>> +        </profile>
>>> +
>>> +        <profile>
>>> +            <!-- An optional Arquillian testing profile that executes
>> tests in a remote JBoss AS instance -->
>>> +            <!-- Run with: mvn clean test -Parq-jbossas-remote -->
>>> +            <id>arq-jbossas-remote</id>
>>> +            <dependencies>
>>> +                <dependency>
>>> +                    <groupId>org.jboss.as</groupId>
>>> +
>> <artifactId>jboss-as-arquillian-container-remote</artifactId>
>>> +                    <scope>test</scope>
>>> +                </dependency>
>>> +            </dependencies>
>>> +        </profile>
>>> +        <profile>
>> 
>> I meant the openshift profile :-)
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/jboss-jdf/jboss-as-quickstart/pull/324/files#r1297983
>> 
> 
> 
> 
> -- 
> Jason Porter
> http://lightguard-jp.blogspot.com
> http://twitter.com/lightguardjp
> 
> Software Engineer
> Open Source Advocate
> Author of Seam Catch - Next Generation Java Exception Handling
> 
> PGP key id: 926CCFF5
> PGP key available at: keyserver.net, pgp.mit.edu
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jboss-jdf/jboss-as-quickstart/pull/324/files#r1297995
pls create an issue and a TODO: for this; we need to do a better design here in the future (as we've already discussed in the past)
These values were borrowed from Qt's similar enum.  They are meant to allow bit-wise combination of vert/horz alignment into a single alignment flag that can be unmasked.
This should be `"WordPress Readme to Markdown"` (note the capital ‘P’ in WordPress).
I'm not all that familiar with the style conventions of this project, but I would write this as:

```scala
private def maybeTranslateName(
    config: SerializationConfig, member: AnnotatedMethod, name: String): String =
  Option(config.getPropertyNamingStrategy)
    .map(_.nameForGetterMethod(config, member, name).getOrElse(name)
```
this is unused as you don't have a typehint
I don't have OpenSSL in my path. Is that a separate tool I need to download? Might be good to link to that page.
I see. In that case, LGTM
Please add a comment about why these counts are different and how you worked them out.
what difference does the order make, why change it? from how i get the doc, numeric arrays are just appended, only if we had keys we would overwrite the first with the second.
alright, appending our thing to the end sounds ok. but the mopa thing should play nicely regardless of the order. i don't really see from their code why it does not work together with this one,, maybe you can ask the authors?
@croby updated in a follow up commit.
Right -- but can Webkit and FF use that also, or does it produce inaccurate results for those browsers?
Почему в угловых скобочках?
You should use Assert.isInstanceOf(..). This way you don't even need to include a message since the it provides a generic and descriptive message
But I like this feature. I figure out the default url by location.href. /v1/metric is defined by our package. When we will update the version, we will also fix the datastream.js. The only thing the user might change is /api/ which is a reasonable default.
https://docs.djangoproject.com/en/dev/topics/i18n/translation/#internationalization-in-javascript-code
If you're using the flag in the template below to conditionally display this, you might as well put the `label` tag directly in the html instead of setting it here.
This might should be "a given DropBox directory" since "DropBox" is a product name.
ai mâncat un spațiu la îmbinarea string-urilor
Should we change this to `form.reset()`?
This runs before the DI extensions are called (which is why it is possible to hook into extensions to register security factories for instance) so there is no service yet
UIScrollView is not required
Does this need a default? Or did you fix the places it gets called?
Le small pas obligé (je trouve que ça fait bizarre à lire, genre bug de css)
I guess it's an inconsistency with the mercurial version of the pom file.
cheers,

Javi



On Thu, Aug 16, 2012 at 12:31 PM, Romain Deltour
<notifications@github.com>wrote:

> In pom.xml:
>
> > @@ -283,6 +319,11 @@
> >                    <groupId>org.codehaus.woodstox</groupId>
> >                    <artifactId>stax2-api</artifactId>
> >                  </artifactItem>
> > +                <artifactItem>
> > +                  <groupId>com.mysql.jdbc</groupId>
>
> I suppose this is not needed for the SSL support ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/daisy-consortium/pipeline-assembly/pull/1/files#r1391597>.
>
>
az zepto e markazi estefade kon
Агам, но литературку я буду делать на последнем этапе. Сейчас важно фактическую точность соблюсти.
Speaking of advanced features "Providing your own id" might belong there as well
Here I would suggest the following;

function downloadThenBootStrapFromURL(url, user, pass) {

  if (url.length < 1) {
    alert("Please input a url to use this feature.");
    return;
  }
  
  if (!user) {
    // No user, no auth
    request.open('GET', url, false);
  } else {
    if (user && !pass) {
      // No pass accept empty string
      // Empty password "" is valid - Basic Auth Standards RFC 2617
      pass = "";
    }
    request.open('GET', url, false, user, pass);
  }


- check for url
- check for user (no user, no auth)
- check no pass but if user add empty pass


below, make sure that the custom button uses the input boxes for all text input values
Yes, that sounds right.  If we have to assign an enumeration, I guess I'd call that 'postsync' headers versus 'streaming' headers.  (And 'presync' for the case where we just load them from the DB then do a refresh.)
Yes I saw that (I looked it up since I have never added the beta tag before). But I don't have anything additional to say, the default beta notice covers that.
If you want to overload the user agent, set Ti.userAgent to whatever you want.
this doesn't make much sense, since this is one event
I  just had a peek at my  work on my machine and it wasn't there.
@jordanbyron: If you don't mind me asking, what kind of object was this originally returning before you called #to_s on it?
Please use non minimized version
not useful comment, should be removed.
I prefer verbosity in this case because the algorithm is not simple
Is there a way to check if this was passed in correctly?
try to remove these later.
remove all links from competition schemas.
(== 0 (mod n divisor)) palauttaa jo sellaisenaan booleanin
this is so that it won't change automatically shortly after you intentionally change it
This isn't the point of this sample. Do you have any specific suggestions?

Also, what part of the code does this apply to?