labofsahil/github-event-dataset-2012 huggingface dataset:
Site updates

* Updated Firefox extension link in the readme
* Added Globberstack support
* Added Drija support

Re the drija support, there's a separate js file for handling it in there. The regexes for matching the page on which to fire a content script are weak - ideally a detection for drija.com/([a-z0-9\-]+)/([0-9]+)/* would be what we'd match, but chrome regex seems to be far more limited. This means the content script gets fired on every drija page, but checks whether a match is found before attempting redirect.

Within drija, they clone not only SO, but askubuntu, serverfault, etc. The queryselector in the drija js is thus slightly looser than the global one to catch the other SE group sites. It *could* probably be merged back in to the one global one without too much heartache, though a big chunk of testing on the other supported sites would be required first to prevent unintended matches getting caught up by it.

Additionally, this pull request only covers the chrome version, safari has not been touched!
dependencies

hmm, looks like an old pull request is getting subsumed into this one...
but anyway... please release? :)

also - https://rt.cpan.org/Ticket/Display.html?id=76946
Custom login redirect and form login disable functionality added to saurus.

Added functionality where you can disable form based logins. Also added an option where you can redirect the admin/editor login to a custom page. That way you can design the page so only ID card based logins are accepted etc.
fix/searchname
Update README.md

added pearescope.com
Some improvements

- after installing from ensemble, if **enabled** from Extensions list, it will update .htaccess rules.
- removed redirect mechanism. Continue reading for discussion.

<br />
About redirect:

I noticed a performance hit on my site: [www.xanderadvertising.com](www.xanderadvertising.com) with the previous version of Language Redirect. The redirect requires the Symphony Engine to initialize and execute the Events attached to a Page before the effective redirect takes place. Adding this to some other factors, there was a penalty of at least 2-3 seconds in page load time.

The changes in this commit will remove the redirection mechanism and instead [detect the language](https://github.com/vlad-ghita/language_redirect/blob/master/lib/class.languageredirect.php#L271-321) as follows (if not found at current point, proceed to next point):

0. On LanguageRedirect initialization, [get from $_REQUEST](https://github.com/vlad-ghita/language_redirect/blob/master/lib/class.languageredirect.php#L199-200).

1. try to get from Cookie
2. try to get from Geolocation Service extension (very useful)
3. try to get from browser's languages
4. try english (this is good in my case. Perhaps you would like to [remove it](https://github.com/vlad-ghita/language_redirect/blob/master/lib/class.languageredirect.php#L314-317))
5. defaults to first supported language-code.

This will comply very well with Search Engines who will index only `www.xanderadvertising.com` instead of `www.xanderadvertising.com/ro/` or `www.xanderadvertising.com/en/`.
Added hasher

(MD5, SHA1, URL Encode/Decode selection)
Set device url

- made device url settable during runtime
- urljoin
- upgraded to robotframework 2.7.4 version
LPS-25253
layout position handling


Layout-model was always overwriting position attribute, small fix for that.


Btw. there is small bug in fixtures now. If fixtures file has "position: 0" it will be ignored and set as max+1. Only positions > 0 are used. I guess right fix for this would be to change database to accept nil values so you could recognize if position is set or not. I didn't figure out easier approach and I wasn't sure if we want to go through migration for such a small bug.
WIP: Changes to recognise upgrade request for RFC 6455

I've made changes so that the upgrade request is now no longer case sensitive as per the advice in https://github.com/superfeedr/ejabberd-websockets/issues/7#issuecomment-3568078.

However, Chrome 16 still does not work. Further changes are required to make this compatible with [RFC 6455](http://tools.ietf.org/html/rfc6455) ([as implemented by FF11 and Chrome 16](http://en.wikipedia.org/wiki/WebSocket#Browser_support)).

Posting this purely as work in progress, but hopefully it will help.

The logs from ejabberd now look like:

    =INFO REPORT==== 2012-01-20 13:38:50 ===
    ERROR: "D(~p:~p:~p) : Request data:~p:~n" - [<0.382.0>,
                                                                         mod_websocket,
                                                                         36,[],
                                                                         {wsrequest,
                                                                          [{"Sec-WebSocket-Version",
                                                                            "13"},
                                                                           {"Sec-Websocket-Key",
                                                                            "j34gc9D6Hl6qnl0Pz7Ai+w=="},
                                                                           {"Sec-WebSocket-Protocol",
                                                                            "xmpp"},
                                                                           {"Origin",
                                                                            "null"},
                                                                           {'Host',
                                                                            "xmpp.local:5288"},
                                                                           {'Connection',
                                                                            "Upgrade"},
                                                                           {'Upgrade',
                                                                            "websocket"}],
                                                                          undefined,
                                                                          false,
                                                                          'GET',
                                                                          ["ws-xmpp"],
                                                                          #Port<0.3190>,
                                                                          gen_tcp,
                                                                          undefined}]

    =INFO REPORT==== 2012-01-20 13:38:50 ===
    D(<0.382.0>:mod_websocket:49) : Origin is valid.

    =INFO REPORT==== 2012-01-20 13:38:50 ===
    D(<0.382.0>:ejabberd_websocket:184) : Websocket!
issue #55: try to enable defaults for all elements without overriding user settings

Hi!

As described in issue #55 new created elements don't get the defaults since setElementsDecorators has not waited for the init method. 

Therefore i tried to enable that behaviour without breaking the file element (the reason the parent constructor is called last).

In my tests it looks alright but i am unsure if i missed something.

If Defaults are disabled the user is able to use their own decorators.

hopefully this does the trick.

bests
ISRC submission support

This implements submit_isrcs. First successful submission is at http://musicbrainz.org/edit/16704180 :sparkles:
Ics

Toro props.
Make HTML code more clear

1. root/styles/prosilver/template/socialnet/user_profile_body.html line 55 - `style="cursor:default;"` should be replaced by class.
2. root/styles/prosilver/template/socialnet/ucp_profile.html - all `<textarea>` elements must have `cols` attribute specified.

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4542
additonal error checking
Поддержка подгрузки на Доброчане

Пока что нет поддержки максимального рейтинга картинок (должен браться из кук). Сейчас табнейлы с рейтингом больше 'r15' скрываются. Дальше будет.
Dust template


It is really good to run test using JHW! However I couldn't use JHW for my latest project because I used [dust.js](http://akdubya.github.com/dustjs/) for template engine.

How about supporting [dust.js](http://akdubya.github.com/dustjs/) from jasmine-headless-webkit?
Many projects will use dust as their template engine since it is really fast and easy to use.
Recently, [dust-rails](https://github.com/thefron/dust-rails) is published and it helped me a lot when I tried to use dust in rails.

This fork used [dust-full-0.3.0.js](https://github.com/akdubya/dustjs/blob/master/dist/dust-full-0.3.0.js) to compile .dust files.
It must be great if jasmine-headless-webkit supports dust.js.

Thank you.
Add some parts from the Nook device and init files.

Change-Id: Ic78780a6f7c75ea11f1fdcb22a3936eeecbc9344
License

Apologies if this is presumptuous of me, but I would feel much more comfortable working with a fork of this project if it had a license. I assumed MIT based on it being the default choice of Rails and related projects.
[pt_BR][_layouts][homepage.html]First translation
raw mode speedup
hardcoded blink for gnome 3.4

starting point for issue #10
fix little typo made twice

from exepcted  to expected ...
Change link in "your code here" box to use the new web interface

Just some effort in pushing migration towards the new web interface. The old link sent you to a page which is more convenient for users of conventional NG readers  though - but I'm not sure if the new interface has such a page.

Maybe the [forum front page](http://forum.dlang.org/) should be revamped a little, currently it has problems such as including the DigitalMars C++ forums and the order in which the forums are listed.

(ping @CyberShadow)
add build script, simplify jar requirements

Hi Corey,

I added a script to build the wisenetlogo.jar, simplified the applet jar requirements to run (only the NetLogo jar is required).

I added the NetLogo and scala-library jars to the repo. This is annoying because they are large but this also makes this repo self-contained. The compilation and testing the example can now all be done with the resources in the repo.

I only needed to specify a class path to the NetLogo jar to compile -- but with the scala-library jar also available there were compilation errors.
主として用語統一結果の反映。メソッド名・クラス名で見出しと使われているものの扱いと単数形・複数形の扱いについては要検討。
Fixes #7

Fixes #7.
Awesome! :)  Did you ever figure out how to handle the progress report?
Does this mean this is no longer needed?: https://github.com/boto/boto/blob/a2bbf639aa3cc9f40029302a5d8b9396852a905a/boto/dynamodb/exceptions.py#L6

Wasn't sure.
chemical1979: you are running my experimental xvba (based on pvr) branch which includes https://github.com/xbmc/xbmc/pull/382. Can you try to duplicate this issue running mainline version?
;lajdg
Note that this is a function, so that whenever it is used, it doesn't create a new anonymous class.  It should probably go in util/ somewhere, but it isn't a big deal.
Even though it's just a warning now?
Nice list of tests :-)
I would like this to be removed and make the attribute related methods in Session public. Would make more sense design wise.
The static ... = 0 adds overhead and flash, storing the zeroes in flash and copying them to .data variables at startup. gcc puts variables with no initializer into .bss and zeros that block efficiently.

Opinions may differ, but mine is we can safely assume uninitialized static variables are zero.
@kuinak I think you're right
oooo, we almost always use exc instead of e, no reason for it though. Is there a valid argument for e?
Ikonky pridane...
Så kan man väl ändå inte göra... :)
Ok, I've made a pass through the commits. To be honest I glazed over the parts that required me to think hard about what was going on. At a glance though things seem fine.
Length should be checked first... No?
I am glad to do this but I should talk with my boss first.
There's no problem, I think.
While upgrading coreutils this morning, I was really sad to find --default-names option disappear from coreutils.  I have a bunch of scripts that rely on GNU coreutils so the aliases file is quite useless.

In my opinion, the best way to solve --default-names for coreutils is to provide a separate bin directory with symlinks to the original commands without the program-prefix.  It won't break any builds by default, and people who need that option can add that directory to their PATH.  Here's my try: [pull request #9818](https://github.com/mxcl/homebrew/pull/9818).  Hope you guys like it.
Upper case MD5 on cowpatty was throwing an audit warning, was that intentional?
Yeah, I think I agree.
We should avoid movn in delay slot. If we are going to support loongson, "movn" would be replaced by Movn macro that consists of several instructions.
System curl won't work here?
Yay I don't know what this means though!
Also, I find the method a bit quite verbose. Maybe something like setDispatcherFirst would be enough? 
Or another idea: setOrderStrategy(DispatcherOrder.DISPATCHER_FIRST) where DispatcherOrder is an enum with two values (proper names should be found)
We pull the derivative on line 61/65 so metrics['read_merged'] is just the value from the last time unit. Thus we can just divide by the time unit to get the per second average for the last interval.

Right? :)
This seems to have broken the behaviour of the Numeric Keypad '/' <VK_DIVIDE> key. See issue 446 (https://github.com/FreeRDP/FreeRDP/issues/446) 
On a German keyboard (-k 0x0407) it now produces a '-' on a UK Keyboard (-k 0x0809) & US (-k 0x0409), shifted it now produces a '?' This should NOT be happening
D'oh. Sorry my fault.
I'm not really sure whether the above lines are required. Do we really have a dependency on tree classes?
We should also do the verification for the `@include(s)` below.
I think this shoud be 

self.toggle_avatar.set_sensitive(force_sensitive_block_button or is_sensitive)
Now that I merged this in I see that it still has some issues yet: http://dl.dropbox.com/u/37982750/sUnT5Pj/right-pane.png http://dl.dropbox.com/u/37982750/sUnT5Pj/right-pane-2.png And it goes totally ape on smaller screens because you set the left position in the css file instead of dynamically: http://dl.dropbox.com/u/37982750/sUnT5Pj/pages-pane.png

Sticking this inside a scroll + resize would be a start and doesn't require you to modify the view:

	var left_pane = $('.one_half:not(.last)').offset();
	var right_pane = $('.one_half.last').offset();
	$('.one_half.last').css( { 'position': 'fixed', 
							   'z-index': 1000, 
							   'top': left_pane.top, 
							   'left': right_pane.left } );

However the code above doesn't account for flowing over the footer if the screen is a little short and it doesn't allow it to scroll up behind the header on short screens too
This is funny. I wasn't getting this error on linux, but when I rebuilt julia (your dlload commit), I started seeing it. I don't get it on mac. Working on it.

-viral



On Jan 28, 2012, at 1:26 PM, JeffBezanson wrote:

> It builds now. Now I get a run time error:
> 
> ```
> /home/jeff/src/julia2/julia/lib/libarpack.so: undefined symbol: dzasum_
> ```
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/2029caaeab85b17e31d0cea53738e47265c05d8b#commitcomment-910574
"liminal" --- great vocab word!!
This can just be `-int32(2147483648)`, no?
Yeah, sorry.  I totally forgot I'd slung that line in there instead of core a while ago.  It's moved over now, and we don't want the feedback mechanism everywhere at the moment.
Why do we time out here? Please explain.
Sure, which pattern is a good example?

I replied to this via email before, which did not work.
> Stopping the analysis means that it is completely aborted

The results got so far are shown. Stopping means we did not check all the files we found.
I think we could do this without passwordLength, couldn't we? just do cbPassword = settings->password_cookie->length -2
Oops, thanks - I'll try another one.
test it and admin announce with <gm> it seems problem of
LANG_SYSTEMMESSAGE_ADMINISTRATOR    = 12000,
that should be 12002 how raynar point 
we put 12002 will check it 
https://github.com/Radicalmangos/Radical/commit/9c637483dd82279ad2ffb250f92fd9d12860205d
possible to add in screen way?

screen -A -m -d -S mangosd ./start core
49376 and 36554
Mhmmm, pistacio's.
I get scared of hardcoding URLs inside `{% trans %}` in fear of making localizers retranslate when the URLs change
could a file ever have `amo.STATUS_NULL`?
смотрите, все ассеты подключаются действительно этим: = javascript_include_tag "application"
а = yield :javascript позволяет в любом месте любой вьюшки написать такое:

-content_for :javascript do
  ....... <- здесь любой код вьюшки

и тогда вот этот любой код постватся вот в то место где написано =yield :javascript
вот про контент_фор: http://apidock.com/rails/ActionView/Helpers/CaptureHelper/content_for

ну собственно осталось только не подключить то что не надо в application.js
yep me too i can't believe somebody would not want that date descending order :)
дефолтным корнем можно считать текущую директорию
* this took you forever *
jk, but fixed this in my own version before you did :P
there is too much as it is
Co to ma być?
I haven't been able to get this to work successfully for iPhone 4.3.  I have a Controller that loads a spinner on "ViewWillAppear", and then a synchronous data call is made on viewDidAppear to load data and remove the spinner. However, the "ViewDidAppear" method is never called when using iPhone 4.3.  It all works perfectly with the iPhone 5.0 simulator.  Are there any new tricks or updates for 4.3?  Thanks!
Ok, es hat sich geklärt. Ich dachte anfangs, dass die Testmethoden unbedingt ein Assert-statement brauchen. Da diese Methode nichts überprüft, soll der Test immer erfolgreich abschließen. In der nächsten Version dieser Datei wird die Zeile gelöscht.
Comparing to true/false is bad style; should be "if not ARGV.build_head?" or "unless ARGV.build_head?"
I think the correct phrase would be "Uma tarefa não pode ser relacionada a uma de suas subtarefas"
I presume you've at least seen [oh-my-zsh](https://github.com/robbyrussell/oh-my-zsh) ?
Horrible typo here: "refreshChlidren"? ;-)
Fixed, js moved to external file.
Hmmm I'd like to get brians thoughts on this one. The bootup of subsystems happens in parallel, so reading another subsystem's model is dangerous.
Вот тут.
Simply a php script for grabbing from the database and then outputs accordingly.
blööd
Fixed in #151
Looks good to me .. thanks JM.
is there any chance it is not declared on this class?
Cool. Already using that plugin. Many thanks.
Great, we would love to make a contribution to a brilliant product.
At some point on the docs we also need to explain the concept of bl-specific "builtins" available in the console.
This is a sample comment on a line.
lady deathwhisper ice bolt cant spell block. so that's too hard!
Now line 371 in `Config.py` won't work, since it depends on it.
This should just be Rails.application no?
what is the file "f" for?
This line failed and this->error was set to true, if the DirectShow filter being used by the OS to render the wave-file didn't support multichannel volume control. The MP3Timer cuts off the just started sound then.
Take care, /tmp may be small, only 58Mb on my DS211j, (and 250Mb on my DS408). You might fill it during the upgrade and lose all the users settings.
Якшы )
I tested it here, and this issue is not happening on my server. Booty Bay to Ratchet is working just fine. I think its probably DB issue, updating YTDB is not usually a good idea.
( jg modification: changed 'outputtracts' from an Enum trait to a Bool trait )
- -m64 will be set automatically if you use ARCH=x86_64, with removing this here the options file will be keept compatible with 32bit builds, the same for  -march=$TARGET_CPU (see config/arch.*)
-mtune is afaik not needed on x86 builds (32bit and 64bit) see http://gcc.gnu.org/onlinedocs/gcc-4.6.2/gcc/i386-and-x86_002d64-Options.html#i386-and-x86_002d64-Options "section -march=cpu-type"
hmmm, good idea.
but IE now asks me to download the json file as soon as the ajax request is done ...
But setting the Content-Type to test/plain fixed all issues, thanks!
So, in my testing, MultiJson wasn't actually loading the Yajl / JSON gem libs, and was defaulting to the ok_json parser. Perhaps by vendoring the lib it broke the default loading somehow? Anyway, I'll be taking this out with a patch later today to remove the vendoring of MultiJSON.
The docs lies.

try {Right(block())} catch {case e => Left(e)}

Means that if the OOME is thrown in Left(e) which will be the case most likely if there's an OOME in block() or Right(), the other blocks won't be executed.
I suggest removal of this method and solving htep roblem in a less wasteful manner.
[maven-dbflute-plugin](http://dbflute.sandbox.seasar.org/maven/plugin/ja/index.html) を plugins に追加します。
schemaName に指定した名前で、DBFluteクライアントディレクトリが作成されます。分かりやすい名前を指定してください。
ну, зачастую достаточно даже так визуально оформить название, чтобы отбить желание использовать метод где-то еще. хотя опять же, главное единый стиль.
Just to check, all this worked because the backward incompatibility was maintained, it was just sending warnings, right :)
I actually added the config structure because I had plans to expand it with other config data, like x/y/z/e scaling (dynamic steps_per_m for easier calibration).  Also, I tried to remove the one "unavoidable" division per calculation by prescaling the divisor to a power of 2 instead of a power of 10 (meters).  I kept the prescaled steps_per_m here in the config structure.  But I had some overflow problem on negative values in my division, so I put it on hold for now.

I wasn't certain of the usage of next_target and move_state (unfamiliar code) so I didn't feel comfortable adding my fields there.  But also I was thinking of the absolute_e value as a config-global setting, not one that you would typically change on the fly.  But I can see how some g-code generator may have different ideas and take liberties with m82/m83 that I'm not prepared to handle.

I've been experimenting with two or three different slicers and they all have their quirks.  Some of them just work better in absolute mode, it seems.  So I've gotten used to adding m82 or m83 as appropriate for each one in my resulting g-code since I created this patch.

I updated update_current_position as rote.  I essentially replaced the MACRO-using logic with variable-using logic across the board.  I don't see any reason to treat one case as special.

    I think M82/M83 should be processed the same way M90/M91 is. 

I think you are right.  I didn't look at that closely, but I understand it more now in the context of this discussion.
Weird. Perhaps cake does extra things to smooth this over? I certainly tried it out with a fresh JVM instance. 

I'll move over to lein2 soon :-)
This causes the test failure of build #272, as the called pattern does not match, so the parameter validator reports an error.
That's temporary workaround - the proper fix required serious rewrite introduced in later commits.
Here it seems to do the right thing in the case of the issue at hand:
    
    >>> len(requests.get('http://dilbert.com').text)
    30294
    >>> type(requests.get('http://dilbert.com').text)
    <type 'unicode'>
Visual C don't understand & on fuctions...
kann man so leider noch nicht verwenden .. deswegen schliesse ich das erstmal .. als issue können wir das gerne aufnehmen ..
you have extra newlines in many files
So... Have the logged_in decorator immediately attempt to acquire a dropbox token if we don't have one already, or the one we have doesn't work? Hm. That makes sense... I think?
https://github.com/robertjwhitney/BeatBox/issues/1
> There is always a timeframe where commits are very dangerous if not
> done correctly.

Yes, the only option would be to have Spartacus pull every other night,
which would result in longer "release cycles" for plugins.
ach....
This will never be true will it?
Esa parte sigue en desarrollo ¬¬
I would say we have to rewrite that file completly, i just made a quick fix (because was not able to install without that information).
parameters are quoted automatically should be mentioned.
yeap I'm making the new realease right now, in a fews hours it's going to be ready for install the new version
:)

Sorry, violates the "conversation with the reader" principle.
org.bukkit.Listener or whatever it is needs to be imported..
wooohooo :)
It's inline cached by V8/Spidermonkey
Vasy si tu veux en attendant ..

Le 30 janvier 2012 18:19, Arthur <
reply@reply.github.com
> a crit :

> ou de cocher le nouveau fichier dans le commit ... Je restaure l'ancienne
> version ???
> Lol, je peut editer ton comment
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cczerty/Robotique-MJC-2011/commit/693f9e0d4e2c231ea3e13ac3ce6ebc43ff3a73b3#commitcomment-914874
>
This isn't quite so simple, unfortunately. do_detail requires a packet, not just a pair_lists_t (it can use packet->code, or packet->src_ipaddr, etc.)  radius_list() returns a pair list, not the parent packet.

It looks like there still needs to be a switch (pair_list), a bit like radius_list, but returning a packet rather than a pair_lists_t (which will obviously, for some values of pair_list, be NULL as there isn't a packet).

Unless there is already a function that does this? I can't find one. I'll see if there is a tidy way to do it.
Fixed since 6c961043b52251b63fb342d14c9cc86a57a028db
i recommend it, we could remove that compatibility after terraria (final?) is released because everyone would need to update all plugins + server bin at that time anyway.
Hi, I am interested in pulling this back to the main line. would you please maybe open a pull request for your change? Thanks
Thanks for this pull request, I've tested it and will merge this now.
This background is non-linear and hard to calculate. Need an updated background. Preferably Vector-based.
Multimedia-student help?
I thought using ';' to terminate a when clause was new 1.9 syntax and 1.8 would barf on it?
You can't do this; you'll close the database before the HTTP request finishes!
Capitalization and punctuation.
It would be an interesting test, to check, what happens, if we are announcing only on IPv6, and listening on that interface.
Too late to be discussed and considered for 1.5.0, possibly for 1.6.0
Added TRY CATCH.  Ran into issue where I fired the timeout, and the Begin would never re-execute.
I have no idea how to set up GitHub XD
Wow, you got around to doing this really fast! I'll check it out and add some CSS to make it fun to click around.
My fault -- did not add the modified files to my repository. Should be:
New: User interface - Small Histogram Chart.cs
Modified: Common - Actions.cs
     Tools - Exporter.cs
     User interface - Controls Market.cs
"VALUES (1, 8, 79684. "Mages " Error, maybe commas =)
I don't personally know what's required for rails 2 sorry, I've only ever used the Rails 3 version.
This will result in small amounts of drift. I.e for a cam with 30 second intervals the first frame is capture at 17:00:00.000, the next at 17:00:30.010, the next at 17:01:00.017 etc. The total drift in a day will depend on the loop interval in App.java and the total number of frames capture.
ska vara "context" med x inte "content"

Samma fel också på nästa rad
It looks like you don't use this property anywhere. Is it necessary?
Be sure to change it back when it is no longer in testing phase :)
No. The changes are incorrect. Final functions are virtual if they override something. Not if they do not.
No such thing as excessive documentation - yours is just fine..;)
This comment is probably an exception though - as the git commit message is dated and says the same. Probably worth removing.
@TomFischer. If the data is not resizable use std::array istead of std::vector.
Nitpick: s/obtained/obtained by/
Would it be simpler to replace the above two lines with the following line:
attrName = attrName(propValue);
cash - Remember that the handler we look for isn't always the first path section. For example `http://my.site/elgg/activity`. You need to do the `str_replace(elgg_get_site_url(), '', $item->getHref())` to get `activity`.
Just to clarify: many of these dependencies are managed now by the parent pom. They were needed when clearpass was maintained as separate extension, and were not removed originally when clearpass was ported to the codebase. This seemed like a good excuse to do some cleanup.
The `type` really is used not only to explain the nature of the error in plain text, but also acts as an identifier that would help with the login flow. For instance, in this implementation, the type is eventually mapped to Spring webflow events such that the "passwordExpired" error type will direct the flow to the password expiration state, etc. 

Plus, it also proposes the advantage that adopters can extend the class and using the same mechanism, create additional new types that could then be mapped to the flow, without a somewhat of an ugly hack. I'd like to leave the option of having an extensible `type` open.

I realize an enum could possibly do the same thing by wrapping the id around itself with a string field but that would limit the extension of LPPE. Not sure what your thought is on this issue. While LPPE supports a number of scenarios OOTB, leaving extension points open for new use cases seems appropriate. 

What do you think?
I have moved the state and related logic to a separate immutable case class, `ClusterHeartbeatSenderState`. Pretty slick. Easier to unit test.
Whoa, nice catch, completely unsuspecting:

scala> import scala.collection.immutable.Stack
import scala.collection.immutable.Stack

scala> Stack.empty eq Stack.empty
res1: Boolean = false
Well, unless the compiler says no, in which case it would be nice to know what breaks it.
Ah, sorry about that, noooow I get it. :(

Good call. But please add a comment about things like that so happy cowboys don't Boy Scout too much in the future.
yes. sweet.
Put explicit method return types on all methods
import PeekMailboxExtension.ack ?
that config option is non-translatable too.

I'm not sure if this is actually the right place and maybe this is just an extension on IPackageController as currently the facets are only relevant to it.  Perhaps this may change over time.
I don't like this. I think there may be some tests that depend on tester not being a sysadmin and this will break those tests. And there's no point in having test users "tester" and "testsysadmin" if they're both sysadmins.
Hrm... this brings up an interesting question. We could reuse the --serve-http and --serve-https config options that the user was already specifying, but they may not necessarily be the same thing.
Can also probably rename to something like "Publish Options". To the end user, the term "config override" is exposing more of the guts of how it's implemented as compared to what it represents.
other way around please - make CONTROL_BT_AUDIO 99 and update CONTROL_END define and control ids in .xml

reason for that is that this will desync skins (button will say Hardware but it will show Audio), while doing other way will just not display audio info but rest would be fine
Yes its possible. I just wanted to keep it as simple as possible and kept it to mimic the old behavior. it can be changed later if someone wants it but I thought to not spend more time than needed on a feature which only a few may use ;)
I agree. Only for display purposes and possibly file copy from http to file
based.
i've added a little utility method for these things a while ago, GetSingleValue(), so you can do this in 1 line
Discussed
if 'iterator' was better named, i would be okay with the name 'util' but right now it looks very confusing. There is an iterator of util(ities) that i'm graphing -- neither of which is really true :(
also the implementation would just be something like this:
Tuple newTuple(List c) {
   Tuple t = newTuple();
   for (int i = 0; i<c.size(); ++i) {
      t.set(i, c.get(i));
   }
}
Not sure if this method should exist
Are you sure there isn't any css rule for warning messages?
Not good enough for several reasons:
* You are using it in a hashSet, so it is really important to have use a good hashCode()
* Consecutive elders are going to have very close hashcodes
* Big startTimestamps are going to hash to the same value, which is really really bad
* A good hashCode is very fast:

    @Override
    public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + (int) (startTimestamp ^ (startTimestamp >>> 32));
        return result;
    }
The same thing I said before about putting ```needs_next_dest_=false``` after the place where you actually set ```next_dest_``` also goes for this log message declaring you've set it. At the time that this log message is printed, it's not actually true yet. Putting these things after the moment in which you actually set next_dest_ avoids confusion if the setting process crashes for some reason.
Publis API? If so, docs for Transport?
fair enough; just add a FIXME so that grep will pick it up
Um, yes. Stupid me... :-)
Mainly to follow actually used Coding Standards ;-)
Actually, looking at the symfony extensions, the only one that could benefit from this optimization is the [YamlExtension](https://github.com/symfony/TwigBridge/blob/master/Extension/YamlExtension.php). Others cannot be marked as consistent as they access external dependencies which may not be in the same state at compile time.
Sorry I meant `js` not `json`
An attempt at something a little original :-) I can remove it if you don't like it.
I'm fairly sure that's only used for ranges. Apple doesn't use it elsewhere.
Ah, I see what I missed now. I added that to the `TUIEdgeInsetsZero` but not the `TUIEdgeInsetsMake()`.
Alright, that makes perfect sense to me. Then how about holding off on this
PR another day, so I can rework the `TUITextRenderer` architecture? That
would save time and effort in the long run.

Or better yet, I'll remove this block for now, and that can be merged in
fine. I'll rework the architecture at a later date.
Strictly speaking, we'll try to load from _a_ jar file, right?  We might conceivably package the native parts in a separate jar.
Are there tabs here? Looks weird.
So unit is the sample spacing. I think that should be made more explicit.
Z_STRLEN_P is not a function but a macro to the string length value stored within the variable structure, so they's no unnecessary computation.

zend_operators.h:

#define Z_STRLEN(zval)			(zval).value.str.len
...
#define Z_STRLEN_P(zval_p)		Z_STRLEN(*zval_p)
запятая на следующую строчку
Any reason this can't go in the initialization above? Other than that, looks good.
I've return explicit block.
`atter_Equal`...
The test could be a bit clearer with some spacer lines before setting new options.
What's an example of a `PolyType` which is a value type?
value map2 is not a member of Option[(Object, java.lang.reflect.Method)]
I just sent a mail to scala-internals describing my changes. I don't have a project for reflect.
To more fully express what I'm thinking: a._1 is a String that came from the user that is consistently the encoded name of the parameter. So far as I know, _._1.name is a name object that is either consistently the decoded name of the parameter or inconsistently either the encoded or the decoded name (which, I'm not sure).
No `$outer` pointers as far as I can see - the class is defined in a top-level companion object:


Compiled from "TrieMap.scala"
public class scala.collection.concurrent.TrieMap$MangledHashing extends java.lang.Object implements scala.util.hashing.Hashing
  SourceFile: "TrieMap.scala"
  Scala: length = 0x
   
  Signature: length = 0x2
   00 27 
  InnerClass: 
   public #45= #42 of #44; //MangledHashing=class scala/collection/concurrent/TrieMap$MangledHashing of class scala/collection/concurrent/TrieMap
  minor version: 0
  major version: 49
  Constant pool:
const #1 = Asciz	SourceFile;
const #2 = Asciz	TrieMap.scala;
...
const #39 = Asciz	<K:Ljava/lang/Object;>Ljava/lang/Object;Lscala/util/hashing/Hashing<TK;>;;
const #40 = Asciz	InnerClasses;
const #41 = Asciz	scala/collection/concurrent/TrieMap$MangledHashing;
const #42 = class	#41;	//  scala/collection/concurrent/TrieMap$MangledHashing
const #43 = Asciz	scala/collection/concurrent/TrieMap;
const #44 = class	#43;	//  scala/collection/concurrent/TrieMap
const #45 = Asciz	MangledHashing;
const #46 = Asciz	scala/util/hashing/Hashing;
const #47 = class	#46;	//  scala/util/hashing/Hashing

{
public int hashCode(java.lang.Object);
  LineNumberTable: 
   line 919: 3
   line 920: 11
   line 921: 16

  LocalVariableTable: 
   Start  Length  Slot  Name   Signature
   0      21      0    this       Lscala/collection/concurrent/TrieMap$MangledHashing;
   0      21      1    k       Ljava/lang/Object;
   11      9      2    hcode       I


  Code:
   Stack=2, Locals=3, Args_size=2
   0:	getstatic	#11; //Field scala/runtime/ScalaRunTime$.MODULE$:Lscala/runtime/ScalaRunTime$;
   3:	aload_1
   4:	invokevirtual	#15; //Method scala/runtime/ScalaRunTime$.hash:(Ljava/lang/Object;)I
   7:	ldc	#16; //int -1640532531
   9:	imul
   10:	istore_2
   11:	iload_2
   12:	invokestatic	#22; //Method java/lang/Integer.reverseBytes:(I)I
   15:	istore_2
   16:	iload_2
   17:	ldc	#16; //int -1640532531
   19:	imul
   20:	ireturn
  LineNumberTable: 
   line 919: 3
   line 920: 11
   line 921: 16

  LocalVariableTable: 
   Start  Length  Slot  Name   Signature
   0      21      0    this       Lscala/collection/concurrent/TrieMap$MangledHashing;
   0      21      1    k       Ljava/lang/Object;
   11      9      2    hcode       I

  Signature: length = 0x2
   00 1E 

public scala.collection.concurrent.TrieMap$MangledHashing();
  LineNumberTable: 
   line 917: 0



  Code:
   Stack=1, Locals=1, Args_size=1
   0:	aload_0
   1:	invokespecial	#37; //Method java/lang/Object."<init>":()V
   4:	return
  LineNumberTable: 
   line 917: 0

  Signature: length = 0x2
   00 21 

}
all those np.array are redudant: Sig, U, Vh are already numpy arrays
ret here will only be for the last map in the composite map. Prehaps it would be better to either return the axes or a list of 'ret's?
I'd suggest removing the extra blank lines around this block of code.
Should this be something like
```
if arg.is_real is True:
    return (yours)
else:
    return self
```
?
I only know how to cheat on this one:


```

>>> a=Indexed('a',i)
>>> Sum(x**i*a(i), (i, 1, n))
Sum(x**i*a[i], (i, 1, n))
>>> _.subs(n,5).doit()
x**5*a[5] + x**4*a[4] + x**3*a[3] + x**2*a[2] + x*a[1]
>>> horner(_)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "sympy/polys/polyfuncs.py", line 168, in horner
    F, opt = poly_from_expr(f, *gens, **args)
  File "sympy/polys/polytools.py", line 3742, in poly_from_expr
    return _poly_from_expr(expr, opt)
  File "sympy/polys/polytools.py", line 3764, in _poly_from_expr
    rep, opt = _dict_from_expr(expr, opt)
  File "sympy/polys/polyutils.py", line 296, in _dict_from_expr
    raise PolynomialError('non-commutative expressions are not supported')
sympy.polys.polyerrors.PolynomialError: non-commutative expressions are not supported
>>> horner(S(str(_)))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "sympy/core/sympify.py", line 178, in sympify
    expr = parse_expr(a, locals or {}, rational, convert_xor)
  File "sympy/parsing/sympy_parser.py", line 162, in parse_expr
    expr = eval(code, global_dict, local_dict)  # take local objects in preference
  File "<string>", line 1, in <module>
TypeError: 'Symbol' object does not support indexing
>>> horner(S(str(_).replace('[','').replace(']','')))
x*(a1 + x*(a2 + x*(a3 + x*(a4 + a5*x))))
```
But everything works perfectly with empty matrices, except for the regression of issue 3401.  There's no need for any special casing (that's the beauty of it).
Just need to update region -> an interface
C'est beau.
Debian/Ubuntu Packager Notice: This change will be IGNORED for Debian and Ubuntu packages, as the command ```bbot-makeconf``` will work on Debian and Ubuntu systems which use the package.
I've added a chore just prior to the go live milestone.
can we do this?
```
private static final GetKeyFunction GET_KEY_FUNCTION...
private static final GroupTupleFunction GROUP_TUPLE_FUNCTION...
```
that's wrong. You cannot pass null.
Shouldn't you check for ENOENT here?
Сюда нужен код такой:

```erb
    <div class="page-header"><h1>Список событий</h1></div>
      <ul class="unstyled">
         <%= render @events %>
      </ul>
    </div>
```
Exactly to address your comment before reagring re-definition.
Your advice is better:
1. allow only one call to Param.parameter for a given parameter)
2. let the file and command line override use the set instance param and not use Param.parameter.

I will implement as you suggest
Isn't it a little weird that the squeel syntax forces us to rename all of our variables (e.g. ````v```` and ````n````)?
Ideally I'd want to refactor the `UserScanningParticipant` to do the work in the DB rather than on the app layer, so at this juncture, I'm tempted to leave the patch as is and raise a separate ticket for this (although I'm not sure in what issue tracker yet). Is there a failing test case for this?
s/stdout/standard output/ 

we can have a standard output term in the glossary if needed?
rephrase /detailed/
How is this relevant to the event?
Added some, is this what you are looking for?
This line should be completely removed. Same applies to other lines of code that used to have only corner spans.
I don't like the fact that phpbb_style is creating its own dependencies. They should be injected.

Instead of `false` for optional values, `null` should be used. False implies a boolean, null is clearer in that regard.
`is_enabled()` should be renamed to just `enabled()`. I'll try to remember to point this out where the methods are defined as well.
You should use initializer here.
```cpp
ImageTexture::ImageTexture() : anisotropyLevel(0) {
    this->initialize();
}
```
Could you please clean this up a bit, to remove the parts which were copied from the XDebug plugin and are not needed here?

For example the `conf_line_prefix` is not needed here and the comment is very misleading.
Building Win8 apps? You'll have 4.5 installed.
用三个等号。
Can you add a comment explaining what this loop is doing?
return -> returns

change -> changes
IMHO, an "else" after an "if" that ends with a "return" is bad style -- It obscures the simple linear main code path with meaningless braces and indent levels.
.2f
missing ``ordering`` field
Yep, it should be, I'll change that.
Nice work here!
+1, gonna revert write_gemspec and do it as you mentionned in another comment
Absolutely. Or we can calculate a shorter hex hash of it?
Would it make sense to add the script to the repository? It could be invoke by the CMake build script.
The pre-steps make sure that there is no duplicates.
They are all GPL or free for comercial use (got from iconfinder and ajaxload.info)
It could be overridden by the caller though. Maybe we should keep it?
Not easily. I know this code is good, I can see all paths it is used and understand them in my head. The other code works over Xcode versions I haven't seen or used in 3 years.
If I do it that way, I get an undefined local variable 'locate_cache'.
Pass `--localstatedir=#{var}` to configure and this will magically work.
Since we're not inspecting the output here, we can avoid forking a subshell:

```rb
system "#{bin}/python", "-c", "import sqlite3"
```
I commented in an old commit. So here again:

```ruby
MacOS.sdk_path 10.7
```
is your lion_prefix.

And because on Xcode-only systems we use the `-isysroot` to set the sdk path, too, you should call `ENV.macosxsdk 10.7` before to avoid having set some other flags and variables. In theory `ENV.macosxsdk 10.7` should already set the `-isysroot` but it does so only for systems that don't have the "Command Line Tools for Xcode"
See [here](https://github.com/mxcl/homebrew/blob/master/Library/Homebrew/extend/ENV.rb#L264)
Ah, right. Fixing.
C'est "git" au sens "nom d'utilisateur / de login"? OK, j'ajoute des quotes, alors.
I consulted with Abe on the name, he suggested .yml.  I originally had .yaml.  I am ok with going back to .yaml.
`get_func_args` does not return defaults (which I am intersted in) but only the name of the arguments, so I can't reuse it. I'm gonna move `get_spec` in scrapy.utils.python and improve it and it's doctests.

`get_func_args` is also untested (formally at least) and does not work with bounded methods (should I fix that in a separate PR?). There's also a global exception handler there whose role I don't understand.
I can't see that comment on github now, did you remove it?

On Mon, Sep 10, 2012 at 6:24 PM, Alexandru Cepoi
<notifications@github.com>wrote:

> In scrapy/contracts/default.py:
>
> > +# contracts
> > +class UrlContract(Contract):
> > +    """ Contract to set the url of the request (mandatory)
> > +        @url http://scrapy.org
> > +    """
> > +
> > +    name = 'url'
> > +
> > +    def adjust_request_args(self, args):
> > +        args['url'] = self.args[0]
> > +        return args
> > +
> > +class ReturnsContract(Contract):
> > +    """ Contract to check the output of a callback
> > +        @returns items, 1
> > +        @returns requests, 1+
>
> @returns requests 2 - returns at least two requests
>
> -- wouldn't this be considered unexpected behaviour?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scrapy/scrapy/pull/167/files#r1570864>.
>
>
Do I need to make a new pull with the change or is there a way for me to update this commit?
In conjunction to the below change, we should delete the application part (like in line 34) afterwards as to allow:
setting = { applications: { ... }, foo: { applications: {...} }
@dsaff 
makes sense
Why here and not in `DOM.events.click`?
Looks like `10` was for development. I imagine Meetup could be `3600` or even `86400`.
There are a number of calls to idParse throughout the code and it's a relatively expensive operation to repeat so many times. Why not have an data type with the parsed out info?
Why not? if/for/else/while etc... statements get this space? Why can't the parameters get it?
Neat.

That mixin is in `_layout.scss`, not `_mixins.scss`. Any reason for the separation?
As I mentioned in previously, variable assignments should be done at the top of the mixin and component.
Yep, changed to "updateInfo" to match the new function name.
We've been using ``window.setTimeout`` to minimize our use of globals in our jslint config.
Yes. I'll change the existing binding case to use the most recent binding.
I'm not super familiar with this gesture, but I found similar commands in Sublime and Eclipse.  In both of those, the command preserves any existing selection and _expands_ it -- making sure the start of the selection is col 0, and moving the end of the selection to col 0 of the line _after_ where the selection used to end (which is really the end of the previous line -- see above).  A nice side effect of this is that you can hit Ctrl+L repeatedly to keep expanding the selection by one line.
This code implies that all children of editor-holder are containers which are stacked vertically. Currently they are, but someone could add a new element that doesn't play by that rule. At the very least, you should state this in a comment in the main-view.html file. You could also verify this assumption programmatically, but that may impact performance.
The newlines here don't actually affect the formatting (since this is HTML). I think it's fine to just have the sentence right after the previous sentence (with just a space character between them).
This could potentially become a place for URL rewriting, should that be necessary in a hosted version. But it probably won't be used consistently anyway and it's too speculative to make part of an official API, I agree.
This works for me with just $("body")--applying it to a and .CodeMirror-lines doesn't seem necessary given the !important declaration.
Again, callback instead of event?
Ah sorry, It was a mistake, parser is able to do this, I've just try
Oh man, start_with reads like broken english. I think I'd rather just use a regex here.